Skip to content

feat: central/shared kubernetes platform landing zone and optional namespace creation in app landing zone#39

Open
lweberru wants to merge 6 commits into
mainfrom
feat/lz-namespace-service-observability-demo
Open

feat: central/shared kubernetes platform landing zone and optional namespace creation in app landing zone#39
lweberru wants to merge 6 commits into
mainfrom
feat/lz-namespace-service-observability-demo

Conversation

@lweberru

@lweberru lweberru commented Jun 12, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Summary

This PR adds a complete Landing Zone namespace service demo flow with real observability integration and Grafana dashboard provisioning.

Included

  • Adds platform-kubernetes module wiring and outputs
  • Adds namespace service demo resources and dashboard definition/import workflow
  • Adds observability outputs and credentials wiring for Grafana/metrics push
  • Adds required provider updates (null) and root/module variable/output wiring

Validation

  • tofu validate (success, with deprecation warnings from provider attributes)

Issue References

Closes #35
Closes #36

Out of Scope

@lweberru lweberru changed the title feat: landing-zone namespace service demo with observability feat: central/shared kubernetes platform landing zone and optional namespace creation in app landing zone Jun 12, 2026
mahauber
mahauber requested changes Jun 15, 2026
View reviewed changes
Comment thread src/modules/platform-kubernetes/3-cluster.tf Outdated
Comment thread src/modules/platform-kubernetes/variables.tf Outdated
Comment thread src/variables.tf Outdated
Comment thread src/namespace-service.tf Outdated
Comment thread src/namespace-service.tf
Comment thread src/outputs.tf
Comment thread src/providers.tf

@mahauber mahauber Jun 15, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this doesnt make sense to me

Comment thread src/providers.tf Outdated
Comment thread src/variables.tf Outdated
Comment thread src/variables.tf
@lweberru

lweberru commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator Author

All review points were addressed in code. The only non-finalized platform capability is provider-native extensions.dns.gatewayApi; until available, we use a Terraform-managed DNS record-set bridge (stackit_dns_record_set) documented in Issue 41.

@lweberru lweberru mentioned this pull request Jun 22, 2026
Open
@lweberru

lweberru commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator Author

@copilot resolve the merge conflicts in this pull request

Copilot AI commented Jul 22, 2026

Copy link
Copy Markdown

@copilot resolve the merge conflicts in this pull request

Resolved and merged the current main into this PR in 0cdb6e9; provider constraints were aligned with the updated base branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mahauber mahauber mahauber requested changes
@dweezl dweezl Awaiting requested review from dweezl dweezl is a code owner
@simpe00 simpe00 Awaiting requested review from simpe00 simpe00 is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

3 participants

@lweberru @mahauber